[PATCH] D78517: [PC Relative][Expolitation] Remove "unskipableSimplifyCode()" in PPCMIPeephole.cpp

[Anil Mahmud via Phabricator via llvm-commits]

anil9 added inline comments.
Herald added a subscriber: wuzish.


================
Comment at: llvm/lib/Target/PowerPC/PPCMIPeephole.cpp:128
     initialize(MF);
-    // FIXME: This introduces another complete traversal of the instructions
-    // in the function in the common case (function is not skipped). Although
-    // this is less than ideal for compile time, this code will go away once
-    // our PC-Rel implementation is complete.
-    unskipableSimplifyCode();
+    // At this point, R2 can be only used as the TOC pointer, which should not
+    // be found when using PC Relative.
----------------
nit : For both the comment and the message in the assert. Ending the sentences with just "PC Relative" seems incorrect, may be use PC Relative instructions/functionality/operations. I see many asserts ending with or without full stops and some with exclamation sign. Not sure which one is correct.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78517