[PATCH] D140108: [AMDGPU][AsmParser][NFC] Refine parsing cache policy modifiers.

Ivan Kosarev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 16 03:02:09 PST 2022


kosarev added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:6047
+    bool Disabling;
+    unsigned P = parseCPolId(getId(), Mnemo, Disabling);
+    if (!P)
----------------
dp wrote:
> Maybe `CPolId` would be better than `P`.
What if just `CPol` then? So it doesn't look like we parse an id into another id.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140108/new/

https://reviews.llvm.org/D140108



More information about the llvm-commits mailing list