yaxunl marked an inline comment as done.
yaxunl added inline comments.
================
Comment at: clang/lib/Basic/Targets/AMDGPU.cpp:414
+ assert(CanonName.startswith("gfx") && "Invalid amdgcn canonical name");
+ Builder.defineMacro(Twine("__") + Twine(CanonName.drop_back(2).upper()) +
+ Twine("__"));
----------------
tra wrote:
> Without an example of the input we're dealing with here it was not obvious
> that the idea here is to convert `gfx<XYY>` -> `__GFX<X>__`. I'd change the
> comment a bit : `e.g gfx901 ->__GFX9__, gfx1033 -> __GFX10__, etc.`
will do when committing
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125909/new/
https://reviews.llvm.org/D125909
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
