[PATCH] D53739: InstCombine: Avoid introducing posion values when lowering llvm.amdgcn.[us]bfe

[Bas Nieuwenhuizen via Phabricator via llvm-commits]

bnieuwenhuizen added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:3452
+    // The case of Width == 0 is handled above, which makes this tranformation
+    // safe.  If Width == 0, then the ashr and lshr instructions become posion
+    // value since the shift amount would be equal to the bit size.
----------------
Small nit in this comment and the commit title/message: posion -> poison?


Repository:
  rL LLVM

https://reviews.llvm.org/D53739